Skip to content

Content&report features#27

Merged
rogargon merged 3 commits into
mainfrom
Content&ReportFeatures
May 7, 2026
Merged

Content&report features#27
rogargon merged 3 commits into
mainfrom
Content&ReportFeatures

Conversation

@EricksonHR

Copy link
Copy Markdown

Added deelte features

private Project project;*/

@ManyToMany
@JoinTable(

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need to configure JoinTable or JoinColumn, use the defaults unless you are using a legacy database

@@ -39,7 +27,10 @@ public class Content {
@ManyToOne
@JoinColumn(name = "user_id")

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not needed like JoinTable

private User user;


@OneToMany(mappedBy = "content", cascade = CascadeType.REMOVE, orphanRemoval = true)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Avoid using the OneToMany side, just the ManyToOne. So, in reports, keep track of the Content corresponding to a Report. And for the reverse direction, use a findByContent in the Reports repository

@github-project-automation github-project-automation Bot moved this from 📋 Backlog to 🏗 In progress in MyPortfoliosGEIADE May 7, 2026
@rogargon rogargon merged commit 18f29dc into main May 7, 2026
1 check passed
@github-project-automation github-project-automation Bot moved this from 🏗 In progress to ✅ Done in MyPortfoliosGEIADE May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants